Skip to content

feat: упростить распределение экспертов и правку предпросмотра - #493

Merged
TThanos3000 merged 2 commits into
devfrom
feat/dev-distribution-review-20261023
Oct 22, 2026
Merged

TThanos3000 merged 2 commits into
devfrom
feat/dev-distribution-review-20261023

Conversation

@TThanos3000

@TThanos3000 TThanos3000 commented Oct 22, 2026 •

Copy link
Copy Markdown
Contributor

Что изменено

Вкладка «Распределение экспертов» показывает одну кнопку «Требуют внимания» только при нехватке экспертов; она включает фильтр проблемных проектов. Убраны сводка всего/показано/успех, блоки сданных черновиков и повторяющаяся фраза про отдельного эксперта. Плашка выбранных проектов белая, с непрозрачными кнопками и ровным расположением.

Автораспределение стало компактнее: на первом шаге только кейсы, которым нужны назначения, с предвыбранными подходящими экспертами. На втором виден полный состав проекта, а не только новые добавления: существующие отмечены «Уже назначен», новые — «Добавится». Нагрузка свернута. Есть «Другой вариант» и ручная правка состава до подтверждения. Под ручным способом возвращена кнопка «К проектам».

Реализация

Experts template/styles/component and additive AllocationQuery fields. При ручной правке сохраняются остальные предложения плана. Существующие назначения в предпросмотре заблокированы. Ошибка пересчета сохраняет старый план и выбор.

API

Зависит от api #815, API deploy первым. Необязательные seed и assignment_overrides для существующего preview endpoint; подтверждение использует только подписанный сервером planToken. Ответ API без изменений.

UI

Сохранены Mont, тонкие кнопки, общие поля и центральные модалки. Desktop/mobile: 1440/768/600/390/360. Предпросмотр с одним существующим и одним новым экспертом показывает 2 из 2. Успешное распределение убирает предупреждение.

Permissions

Не менялись. API проверяет роль, программу, собственную команду, кейс, пул экспертов, лимит, уникальность и revision. Предпросмотр и редактирование не назначают экспертов до confirm.

Проверки

  • 35 Experts tests, 54 relevant PostgreSQL API tests.

  • build:pr; global ESLint 0 ошибок/6 прежних предупреждений.

  • Targeted Stylelint, exact committed Git blob Prettier, diff --check.

  • Browser QA 768/600/360/390/1440, визуальный просмотр desktop/phone.

  • Включение attention-фильтра, белая ровная плашка, возврат к проектам, выбор пула.

  • Полный состав, другой вариант, редактор и новый signed preview без записи.

  • Confirm/read-back на synthetic fixtures 360/390/1440: все проекты получили нужный состав, старые назначения сохранились.

  • 0 JS/server ошибок и 0 писем; реальные пользователи не менялись.

  • DEV deployment 39866467482: 447 suites / 2421 tests, format/lint/build/rsync passed.

  • Published DEV assets browser QA 1440/390 + isolated API: full composition, reroll, per-project edit, confirmation/read-back passed; 0 JS/server errors, 0 mail.

  • Real DEV anonymous GET smoke: 4 routes passed; task-owned QA and dedicated PostgreSQL stopped.

Acceptance criteria

  • Упрощение вкладки и обоих шагов распределения.
  • Предупреждение только при проблеме и работает как фильтр.
  • Полный состав и возможность изменить будущие назначения.
  • Безопасность текущих назначений/оценок и серверное подтверждение.
  • Адаптивная белая плашка.

Не проверено

Опубликованные DEV assets проверены с изолированным API. Изменение реальных пользователей и отправка реальных писем исключены из QA.

Риски / migration

Миграции не нужны. API #815 выкладывается до этого PR. Production вне области задачи.

@TThanos3000
TThanos3000 merged commit e3ca7de into dev Oct 22, 2026
@TThanos3000
TThanos3000 deleted the feat/dev-distribution-review-20261023 branch October 22, 2026 22:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants